Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump zipkinSenderVersion to version 3.4.1 #3427

Merged
merged 1 commit into from
Sep 11, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 11, 2024

Bumps zipkinSenderVersion from 3.4.0 to 3.4.1.
Updates io.zipkin.reporter2:zipkin-reporter-brave from 3.4.0 to 3.4.1

Release notes

Sourced from io.zipkin.reporter2:zipkin-reporter-brave's releases.

Zipkin Reporter 3.4 deprecates AsyncReporter/SpanHandler queuedMaxBytes and disables it by default.

When introduced, AsyncReporter had three ways to trigger a queue flush:

  • queuedMaxSpans - when the number of spans in the queue exceeds a threshold
  • queuedMaxBytes - when the size of the spans in the queue exceeds a threshold
  • messageTimeout - when a span has been in the queue longer than a threshold

queuedMaxBytes was deprecated because requires time in the critical path, to calculate the size of a span to make sure it doesn't breach the threshold. This is problematic in tools that check for pinning, like Virtual Threads.

Thanks a lot to @​reta for sorting this out!

Full Changelog: https://github.com/openzipkin/zipkin-reporter-java/compare/3.3.0..3.4.1

Commits
  • 151238b [maven-release-plugin] prepare release 3.4.1
  • ad5b40f Routine dependency updates (#271)
  • 302b54c Avoid encoding list of spans twice in BaseHttpSender (#270)
  • 5191332 Adds SECURITY.md and scanning workflow (#267)
  • 3af6593 [maven-release-plugin] prepare for next development iteration
  • See full diff in compare view

Updates io.zipkin.reporter2:zipkin-sender-urlconnection from 3.4.0 to 3.4.1

Release notes

Sourced from io.zipkin.reporter2:zipkin-sender-urlconnection's releases.

Zipkin Reporter 3.4 deprecates AsyncReporter/SpanHandler queuedMaxBytes and disables it by default.

When introduced, AsyncReporter had three ways to trigger a queue flush:

  • queuedMaxSpans - when the number of spans in the queue exceeds a threshold
  • queuedMaxBytes - when the size of the spans in the queue exceeds a threshold
  • messageTimeout - when a span has been in the queue longer than a threshold

queuedMaxBytes was deprecated because requires time in the critical path, to calculate the size of a span to make sure it doesn't breach the threshold. This is problematic in tools that check for pinning, like Virtual Threads.

Thanks a lot to @​reta for sorting this out!

Full Changelog: https://github.com/openzipkin/zipkin-reporter-java/compare/3.3.0..3.4.1

Commits
  • 151238b [maven-release-plugin] prepare release 3.4.1
  • ad5b40f Routine dependency updates (#271)
  • 302b54c Avoid encoding list of spans twice in BaseHttpSender (#270)
  • 5191332 Adds SECURITY.md and scanning workflow (#267)
  • 3af6593 [maven-release-plugin] prepare for next development iteration
  • See full diff in compare view

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps `zipkinSenderVersion` from 3.4.0 to 3.4.1.

Updates `io.zipkin.reporter2:zipkin-reporter-brave` from 3.4.0 to 3.4.1
- [Release notes](https://github.com/openzipkin/zipkin-reporter-java/releases)
- [Changelog](https://github.com/openzipkin/zipkin-reporter-java/blob/master/RELEASE.md)
- [Commits](openzipkin/zipkin-reporter-java@3.4.0...3.4.1)

Updates `io.zipkin.reporter2:zipkin-sender-urlconnection` from 3.4.0 to 3.4.1
- [Release notes](https://github.com/openzipkin/zipkin-reporter-java/releases)
- [Changelog](https://github.com/openzipkin/zipkin-reporter-java/blob/master/RELEASE.md)
- [Commits](openzipkin/zipkin-reporter-java@3.4.0...3.4.1)

---
updated-dependencies:
- dependency-name: io.zipkin.reporter2:zipkin-reporter-brave
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: io.zipkin.reporter2:zipkin-sender-urlconnection
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the type/dependency-upgrade A dependency upgrade label Sep 11, 2024
@violetagg violetagg changed the title Bump zipkinSenderVersion from 3.4.0 to 3.4.1 Bump zipkinSenderVersion to version 3.4.1 Sep 11, 2024
@violetagg violetagg added this to the 1.2.0-RC1 milestone Sep 11, 2024
@violetagg violetagg merged commit 5231065 into main Sep 11, 2024
14 checks passed
@dependabot dependabot bot deleted the dependabot/gradle/main/zipkinSenderVersion-3.4.1 branch September 11, 2024 13:03
violetagg added a commit that referenced this pull request Sep 11, 2024
@violetagg violetagg added status/superseded An issue that has been superseded by another ignore-changelog labels Sep 24, 2024
@violetagg
Copy link
Member

This is superseded by #3438

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-changelog status/superseded An issue that has been superseded by another type/dependency-upgrade A dependency upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant